Set tests to run on 'dev' branch in workflow - #109
Merged
Merged
Conversation
dev will now be used before main
Contributor
There was a problem hiding this comment.
Pull request overview
Updates contributor guidance and CI configuration so pull requests targeting the new dev staging branch run the test workflow, aligning the repository process where dev is used before main.
Changes:
- Documented the new contribution workflow: feature work targets
dev, withmainreserved for downstream merges and the tag-scan exception. - Updated the test GitHub Actions workflow to run on PRs to both
devandmain. - Updated the CI workflow documentation table in
AGENTS.mdto reflect the new trigger branches.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
AGENTS.md |
Adds a “Contributing Workflow” section and updates the CI workflow table to include dev PRs for tests. |
.github/workflows/test.yml |
Expands the pull_request trigger branches to include dev alongside main. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
egrace479
added a commit
that referenced
this pull request
Jul 8, 2026
dev will now be used before main * Clarify Agent instructions for dev workflow
NetZissou
added a commit
to ABC-Center/catalog
that referenced
this pull request
Sep 1, 2026
* Set tests to run on 'dev' branch in workflow Pull from Imagomics catalog [PR 109](Imageomics/catalog#109) Excluded agent md updates from the original commits. dev will now be used before main (cherry picked from commit 9a4b28089fe1711521e0f736603d5de74b5f1a5f) * Allow for HF org with different name from code platform org Pull from Imageomics catalog [PR 110](Imageomics/catalog#110) Excluded agents md updates from the original commits. * Allow for HF org with different name from codebase platform * Add terminal rendered error message in case of missing/incorrect config keys at Vite start * Improve validation (cherry picked from commit: 2f1f18541e2e769f7ada5f45ee4dcb4beb42a314) * Modularize the codebase Pulled from Imageomics catalog [PR 115](Imageomics/catalog#115) Excluded agents md updates from the original commit. * Move type/platform-based repo fetching to dedicated api modules * Check releases are loaded before fetching code repos * Move rendering, URL management, and UI logic to external modules * Move tag lookup creation to tag-related utils file to reduce parameter passing internal variable only required for normalizing tags * Restructure repo to accommodate larger number of source files arrange by purpose * Update AGENTS repository layout description to reflect changes initial description updates generated with Gemini 3.1 Pro in a modularization discussion thread updated paths for test list * Add tests for new API fetch modules and URL management Tests constructed with aid of Gemini 3.1 Pro and Thinking Switch to jsdom testing strategy, copilot suggestion * Move HF default tag filtering to normalizeTag utility consolidates tag processing, creating single location for potential personalization of tags cherry-picked from commit: 76a1f4a16cdc1ba2e74d237d92e4ebc66c0b709c * Update packages to remove vulnerabilities Pulled from Imageomics catalog cherry-picked from commit: 939bfeb4f5b750fb74e9acfb08fa4c03673a3253 use named imports, error with jsyaml, standardize fs use * Update public/config.yaml add `ORGANIZATION_NAME` field in `public/config.yaml` and set value as "ABC-Center" Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com> * Add Codeberg as a code platform option Pulled from Imageomics catalog [PR 121](Imageomics/catalog#121) cherry-pick commit: 8315b22a4ad8106b93473d3c5c748fdac0697c97 * Switch to local SVG use for code platform logo Codeberg logo is too complex for simple svg path definition * Check release not undefined for ribbon, as with stars and forks, there may not be a release * Generalize tests for code fetching to cycle platforms Add profile repo key, Codeberg uses .profile like GitHub has .github * Struct/platform keys Pulled from Imageomics catalog [PR 126](Imageomics/catalog#126) cherry-pick commit: 1a5e7bad2f362acdcaee06802ba7e541ff909354 * Consolidate platform-specific non-ui value fetches in dedicated file Reduces platform-specific edits required to token-handling in scripts * Remove duplicated key from return fallback defined at UI render * Revise data mapping test * Clarify demo (HF space) jargon Pulled from Imageomics catalog [PR 132](Imageomics/catalog#132) cherry-pick commit: 7fb4eff9172509f30056ce0c8076560bb9a02a96 * Update terminology to be more general (use 'demos' instead of 'spaces') * Update documentation where relevant to match * removed contribution file & agent file, preserved ABC customization while changing "space" to "demo" * Add GitLab as a code repository platform option Pulled from Imageomics catalog [PR 134](Imageomics/catalog#134) cherry-pick commit: c5ce9487905f266448c33398025f678ea75ab825 * Update documentation to reflect increased code platform options * ensure no private repos leak * Use platform-specific token auth in release fetch script * Move build-time helper functions to scripts utility * Make org name testing more robust based on platform differences * make build script requests more efficient and respectful * Include projects from subgroups, but exclude shared projects for GitLab consistent with the stated goal of the catalog and the way in which GitLab presents a group's projects * Add number of results display near search bar Pulled from Imageomics catalog [PR 135](Imageomics/catalog#135) cherry-pick commit: 6933942875119ea87a1624cd80647032ceda30f0 * feat: display result count next to search bar, adjusts with all filters Co-authored-by: egrace479 <38985481+egrace479@users.noreply.github.com> * match formatting standard * Add polite screen-reader notice of result count --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: egrace479 <38985481+egrace479@users.noreply.github.com> * Use config platform name for logging process Pulled from Imageomics catalog [PR 139](Imageomics/catalog#139) cherry-pick commit: 51800ecaa94cd7698b57b6e5f6d0f1629581b219 Hugging Face fetch is hard-coded as 'HF', and pulling from display is incompatible with the SVG use in a Node environment * Apply batched suggestions from code review Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com> --------- Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com> Co-authored-by: egrace479 <e.campolongo479@gmail.com> Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
devwill now be used beforemain. I addeddevto the main branch protection rule and set both to squash or rebase merge. The workflow will be mostly squash intodevand rebase intomain, asdevwill be used for staging changes.I also amended the
AGENTS.mdto reflect this change.Closes #108